Skip to content

[mlir][dxsa] Add dfma instruction#208

Open
dzhidzhoev wants to merge 1 commit into
vshiryaev/dxsa-mlir-double-arithmetic-instructionsfrom
users/dzhidzhoev/mlir/fdma
Open

[mlir][dxsa] Add dfma instruction#208
dzhidzhoev wants to merge 1 commit into
vshiryaev/dxsa-mlir-double-arithmetic-instructionsfrom
users/dzhidzhoev/mlir/fdma

Conversation

@dzhidzhoev

@dzhidzhoev dzhidzhoev commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Example:
dxsa.dfma , r<0>, r<1>, r<2>
dxsa.dfma_sat , r<0>, r<1>, r<2>

Stacked on #190.

Example:
dxsa.fdma <r0>, r<0>, r<1>, r<2>
dxsa.fdma_sat <r0>, r<0>, r<1>, r<2>
@tagolog tagolog changed the title [mlir][dxsa] Add fdma instruction [mlir][dxsa] Add dfma instruction Jun 28, 2026
@tagolog

tagolog commented Jun 28, 2026

Copy link
Copy Markdown

Commit message contains wrong instruction name fdma. Should be dfma.

#define MLIR_DIALECT_DXSA_IR_DXSADOUBLEARITHOPS

include "mlir/Dialect/DXSA/IR/DXSAOpBase.td"
include "mlir/Dialect/DXSA/IR/DXSAFPArithOps.td"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we just move DXSA_MultiplyAddOp to the DXSAOpBase.td? This seems a better solution rather then including FP ops to double.

@dzhidzhoev dzhidzhoev Jun 28, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, that makes sense. I did that in #210. I will update it here after #210 and #190 get merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants